Skip to content

Fix tabs rendering in Contributor Guide#617

Merged
mfisher87 merged 2 commits intogeojupyter:mainfrom
pblottiere:doc_fix_tabs
Apr 8, 2025
Merged

Fix tabs rendering in Contributor Guide#617
mfisher87 merged 2 commits intogeojupyter:mainfrom
pblottiere:doc_fix_tabs

Conversation

@pblottiere
Copy link
Contributor

@pblottiere pblottiere commented Apr 8, 2025

Description

Hello 👋,

I started using JupyterGIS because I was curious about how it works with QGIS, and I found a small typo in the documentation.

Before this fix on the online documentation:

image

With this correction:

image

Checklist

  • PR has a descriptive title and content.
  • PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
  • PR has one of the labels: documentation, bug, enhancement, feature, maintenance
  • Checks are passing.
    Failing lint checks can be resolved with:
    • pre-commit run --all-files
    • jlpm run lint

📚 Documentation preview: https://jupytergis--617.org.readthedocs.build/en/617/
💡 JupyterLite preview: https://jupytergis--617.org.readthedocs.build/en/617/lite

@github-actions
Copy link
Contributor

github-actions bot commented Apr 8, 2025

Binder 👈 Launch a Binder on branch pblottiere/jupytergis/doc_fix_tabs

@mfisher87
Copy link
Member

Oh shoot, great catch! Thank you 🙏

@mfisher87 mfisher87 added the documentation Improvements or additions to documentation label Apr 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Apr 8, 2025

Integration tests report: appsharing.space

@mfisher87
Copy link
Member

Thanks again for catching this @pblottiere 💯

@mfisher87 mfisher87 merged commit d9a63f0 into geojupyter:main Apr 8, 2025
14 of 15 checks passed
HaudinFlorence pushed a commit to HaudinFlorence/jupytergis that referenced this pull request Jan 28, 2026
* Finish the first tab

* Correct admonition syntax
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants